Skip to content

Conversation

@xenolinux
Copy link
Contributor

@xenolinux xenolinux commented Apr 11, 2022

Bug
Preview
OCP version: 4.6+
QE contact @sunilcio

@openshift-ci openshift-ci bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Apr 11, 2022
@netlify
Copy link

netlify bot commented Apr 11, 2022

Deploy Preview for osdocs ready!

Name Link
🔨 Latest commit c6d7145
🔍 Latest deploy log https://app.netlify.com/sites/osdocs/deploys/6255549450e5f7000945b7eb
😎 Deploy Preview https://deploy-preview-44431--osdocs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@xenolinux xenolinux changed the title BZ2053605: Add instructions on how to scale up AWS nodes [WIP]BZ2053605: Add instructions on how to scale up AWS nodes Apr 12, 2022
@openshift-ci openshift-ci bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Apr 12, 2022
@openshift-ci openshift-ci bot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Apr 12, 2022
@xenolinux xenolinux force-pushed the scale-up-aws branch 6 times, most recently from 07139ec to df6f9ef Compare April 12, 2022 10:29
@xenolinux xenolinux changed the title [WIP]BZ2053605: Add instructions on how to scale up AWS nodes BZ2053605: Add instructions on how to scale up AWS nodes Apr 12, 2022
@openshift-ci openshift-ci bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Apr 12, 2022
@sunilcio
Copy link

@gpei hi, could you help check if your team can review this?

@gpei
Copy link

gpei commented Apr 14, 2022

@sunilcio hi, this should be scaling up via machineset, @sunzhaohua2 could you help to check this?

@sunzhaohua2
Copy link

This is about master instancetype change, we didn't test this before, I can have a try tomorrow and let etcd team help to check, I feel this is not supported.

@sunzhaohua2
Copy link

@xenolinux I added a comment in https://bugzilla.redhat.com/show_bug.cgi?id=2053605#c7, tested with worker machine and master machine, this couldn't change instance type.

@xenolinux
Copy link
Contributor Author

@sunzhaohua2 Does t

@xenolinux I added a comment in https://bugzilla.redhat.com/show_bug.cgi?id=2053605#c7, tested with worker machine and master machine, this couldn't change instance type.

@sunzhaohua2 Does this mean that this change is unsupported? I am in doubt about whether we can add these steps to our documentation?

@sunzhaohua2
Copy link

@xenolinux sorry for the late reply. Yes, I think this is not supported, at least only this change in the doc is unsupported. We will support this feature in 4.12, this is epic https://issues.redhat.com/browse/OCPCLOUD-1372, this is enhancements doc pr openshift/enhancements#1008

@xenolinux
Copy link
Contributor Author

#44431 (comment) closing this PR since this change is unsupported in the docs.

@xenolinux xenolinux closed this Apr 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants